Skip to content

chore(ci): remove docker dependency from deb/rpm package targets#24864

Merged
thomasqueirozb merged 1 commit intomasterfrom
thomas/remove-docker-from-package-targets
Mar 6, 2026
Merged

chore(ci): remove docker dependency from deb/rpm package targets#24864
thomasqueirozb merged 1 commit intomasterfrom
thomas/remove-docker-from-package-targets

Conversation

@thomasqueirozb
Copy link
Contributor

@thomasqueirozb thomasqueirozb commented Mar 6, 2026 •
edited
Loading

Summary

  • Replace $(CONTAINER_TOOL) run with $(VDEV) package deb/rpm in Makefile deb/rpm targets
  • Remove the cargo-deb self-install block from package-deb.sh — version is now managed via prepare.sh/setup action
  • Add cargo-deb: true and cmark-gfm installation to k8s_e2e.yml
  • Add cargo-deb module to prepare.sh invocation in publish.yml (bootstrap-ubuntu-24.04.sh already provides cmark-gfm and rpmbuild)

Vector configuration

NA

How did you test this PR?

CI

Run - make package-deb-x86_64-unknown-linux-gnu passed

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

NA

@github-actions github-actions bot added domain: ci Anything related to Vector's CI environment domain: releasing Anything related to releasing Vector labels Mar 6, 2026
@thomasqueirozb thomasqueirozb marked this pull request as ready for review March 6, 2026 21:44
@thomasqueirozb thomasqueirozb requested a review from a team as a code owner March 6, 2026 21:44
thomasqueirozb
thomasqueirozb commented Mar 6, 2026
View reviewed changes
Copy link
Contributor Author

@thomasqueirozb thomasqueirozb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ci-run-k8s

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Mar 6, 2026
pront
pront approved these changes Mar 6, 2026
View reviewed changes
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good simplification

@thomasqueirozb thomasqueirozb enabled auto-merge March 6, 2026 22:00
@thomasqueirozb thomasqueirozb added this pull request to the merge queue Mar 6, 2026
Merged via the queue into master with commit 70a8bfa Mar 6, 2026
114 of 115 checks passed
@thomasqueirozb thomasqueirozb deleted the thomas/remove-docker-from-package-targets branch March 6, 2026 22:55
@github-actions github-actions bot locked and limited conversation to collaborators Mar 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@pront pront pront approved these changes

Assignees

No one assigned

Labels

domain: ci Anything related to Vector's CI environment domain: releasing Anything related to releasing Vector no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@thomasqueirozb @pront